The latest updates on your projects. Learn more about Vercel for Git ↗︎

@zzuziak Just a quick comment, on this repo (and most of our repo) we use Yarn, this PR shouldn't have a package-lock.json file

Hey @bonustrack, thanks 🙏 I was trying to fetch this branch with npm as a dependency to test it, must have generated the file then.

I can't test the new schema updated on two branches (snapshot.js and hub) - what's your approach to test the repos together before merging PRs?

Updated with a single param for space.voting, currently the setting lies within the SettingsVotingBlock:

To make it more readable I can move it to an individual block and mention that it also allows aliased proposal creation.

@zzuziak You can use yarn link to get Snapshot.js in the UI. To test the whole flow you would need to run the hub locally but I recommend focus on the UI integration, we will do some change on the hub to get settings with the new aliased param accepted

Ok, ready for first review 👌

I didn't use aliasing for the usePersonalSign, I assumed Gnosis should always require the signature but correct me if I wrong, still digging into it 🙈

We should allow alias actions only for these actions: "proposal", "vote" and "delete-proposal"

I think we have a miss understanding, I was referring to this comment from Fabien. I don't see this handled anywhere in your update.

In the current implementation in this PR it will also try to update settings with alias

I think we have a miss understanding, I was referring to this comment from Fabien. I don't see this handled anywhere in your update.

In the current implementation in this PR it will also try to update settings with alias

@samuveth I tagged you in the comments to clarify, let me know if it's still confusing 🙏

Hey @samuveth could you have a look at this PR again and see if there is something left to be done?

@zzuziak yes, there are things to be done outside of this PR first and the useClient code need to be updated since we have depreciated personal sign since this PR was made. I'll take care of it from here and I'll assign it to myself.

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.